Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR - rebased of ibre5041/master #55

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

masovsky
Copy link

@masovsky masovsky commented Dec 9, 2022

No description provided.

@sampath-kolisetty
Copy link
Contributor

@masovsky @vojtechDB

  • Create a separate PR for Log4j version upgrade mentioning the use case/reason for upgrading
  • Create a separate PR for Java version upgrade
  • Regarding support for web behind proxy, please update the ReadME file stating the command to use, use case and the
    steps to test it.

@sampath-kolisetty
Copy link
Contributor

@masovsky
java -Dhttp.nonProxyHosts="github.mycompany.com" -Dhttps.proxyHost=192.168.100.1 -Dhttps.proxyPort=8080 -jar OpsGenieExportUtil-0.xx.xx.jar .......
The above command doesn't work. You will have to modify README mentioning how to run the jar file using above proxy properties.

@idi2022
Copy link

idi2022 commented Jan 24, 2023

Hello,
and thanks for all the edits
can we know when we will have the new version because we are blocked by the proxy from ni internal gitlab
thank you

@ibre5041
Copy link

ibre5041 commented Apr 3, 2023

@masovsky @vojtechDB

  • Create a separate PR for Log4j version upgrade mentioning the use case/reason for upgrading
  • Create a separate PR for Java version upgrade
  • Regarding support for web behind proxy, please update the ReadME file stating the command to use, use case and the
    steps to test it.

The latest Log4J is not available for Java 1.6. That's Why these two patches have to be applied together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants